-
Notifications
You must be signed in to change notification settings - Fork 314
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Denormalize namespace name to job table #934
Conversation
Signed-off-by: henneberger <[email protected]>
e427eac
to
e7f2b45
Compare
Codecov Report
@@ Coverage Diff @@
## main #934 +/- ##
============================================
- Coverage 72.88% 72.57% -0.31%
- Complexity 437 834 +397
============================================
Files 112 167 +55
Lines 2054 3770 +1716
Branches 188 355 +167
============================================
+ Hits 1497 2736 +1239
- Misses 269 568 +299
- Partials 288 466 +178
Continue to review full report at Codecov.
|
@@ -0,0 +1,2 @@ | |||
ALTER TABLE jobs ADD namespace_name VARCHAR(255); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With PR #935 now merged, mind updating the migration version to V22
to maintain ordering?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pr #935 was branched off of this commit so this pr can be closed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good 👍 But, we'll need to update the migration version to V22
to maintain ordering.
The namespace uuid column and constraint can be dropped, and a new constraint on namespace name can be added in the next release